home *** CD-ROM | disk | FTP | other *** search
/ NetNews Offline 2 / NetNews Offline Volume 2.iso / news / comp / lang / c-part1 / 5058 < prev    next >
Encoding:
Text File  |  1996-08-05  |  3.3 KB  |  114 lines
  1. Path: mail2news.demon.co.uk!genesis.demon.co.uk
  2. From: Lawrence Kirby <fred@genesis.demon.co.uk>
  3. Newsgroups: comp.lang.c
  4. Subject: Re: fopen not working
  5. Date: Mon, 12 Feb 96 20:55:53 GMT
  6. Organization: none
  7. Message-ID: <824158553snz@genesis.demon.co.uk>
  8. References: <4fjnj1$4q@fig.leba.net>
  9. Reply-To: fred@genesis.demon.co.uk
  10. X-NNTP-Posting-Host: genesis.demon.co.uk
  11. X-Newsreader: Demon Internet Simple News v1.27
  12. X-Mail2News-Path: genesis.demon.co.uk
  13.  
  14. In article <4fjnj1$4q@fig.leba.net> fweaver@leba.net  writes:
  15.  
  16. >Good evening to all C programers
  17. >
  18. >   I have a problem I thought someone might be able to shed some light on 
  19. >for me. Below is some of the code I wrote for an install program.
  20. >
  21. >
  22. >#include<stdio.h>
  23. >#include<conio.h>
  24.  
  25. There doesn't appear to be anything here that need conio.h. Also it isn't
  26. a standard C header file so is best avoided when posting to comp.lang.c
  27.  
  28. >char buf[125],stwd[110],trwd[110],*pt,*pt1;
  29. >int i,vr,vd;
  30. >FILE *in, *out;
  31. >main()
  32. >{   [snip]
  33. > 
  34. > instlfile("trkr.cf_","trkr.cfg");
  35. > instlfile("maplvl3.da_","maplvl3.dat");
  36. > instlfile("atpctrkr.do_","atpctrkr.doc");
  37. > instlfile("readme.1s_","readme.1st");
  38. > instlfile("file_id.di_","file_id.diz");
  39. > 
  40. > [snip]
  41. >
  42.  
  43. > if((in=fopen(buf,"r"))!=NULL)
  44. >  { do{ fgets(buf,120,in);
  45. >   pt= (char *)strstr(buf,"XXXXX");
  46. >   if(pt!=NULL){ fclose(out); pt=pt+5;if((out=fopen(pt,
  47. >"w"))==NULL)printf("ERROR");
  48. > else{printf("\nInstalling %s",pt);continue; }}
  49. >   fputs(buf,out);
  50. > }while(feof(in)==0);
  51. > }
  52. >}
  53.  
  54. This is almost impossible to read. I've reformatted into something clearer.
  55. It isn't the idea of C to cram a program into as few lines as possible.
  56.  
  57. >>    if ((in=fopen(buf,"r")) != NULL) {
  58. >>        do {
  59. >>            fgets(buf,120,in);
  60.  
  61. What is 120? If you want to pass the size of buf then use sizeof buff, or
  62. perhaps better define a constant and use that in both places.
  63.  
  64. fgets can fail and return NULL which you need to test then and there. This
  65. sort of loop normally takes the form:
  66.  
  67.           while (fgets(buf, sizeof buf, in) != NULL) {
  68.             ...
  69.           }
  70.  
  71. >>            pt = (char *)strstr(buf,"XXXXX");
  72.  
  73. Why the cast? Did you include string.h? If not your compiler probably produced
  74. a warning. The cast may stop the warning being generated but the compiler
  75. will still generate code as if strstr() returned an int.
  76.  
  77. >>            if (pt != NULL) {
  78. >>                fclose(out);
  79.  
  80. You'd better make sure that out contains a valid file pointer value e.g.
  81. NULL is not legal here. Check especially the first pass through the loop.
  82.  
  83. >>                pt=pt+5;
  84.  
  85. The normal C idiom is pt += 5;
  86.  
  87. >>                if ((out=fopen(pt, "w")) == NULL)
  88.  
  89. That looks OK. Are you sure it is exactly what you have in your code e.g.
  90. not == instead of =?
  91.  
  92. >>                    printf("ERROR");
  93. >>                else {
  94. >>                    printf("\nInstalling %s",pt);
  95. >>                    continue;
  96. >>                }
  97. >>            }
  98. >>            fputs(buf, out);
  99. >>        } while (feof(in)==0);
  100.  
  101. It is very rarely that you need to use feof in C. Anyway this is far too
  102. late since by the time you've detected that the line hasn't been read in
  103. you've already tried to process it. The while (fgets() != NULL) loop above
  104. is the correct approach.
  105.  
  106. >>    }
  107. >>}
  108.  
  109. -- 
  110. -----------------------------------------
  111. Lawrence Kirby | fred@genesis.demon.co.uk
  112. Wilts, England | 70734.126@compuserve.com
  113. -----------------------------------------
  114.